Skip to content

Conversation

@swaldmann
Copy link
Contributor

@swaldmann swaldmann commented Jul 25, 2025

This PR creates a staging deployment for pushes on main. I also set org-wide defaults for CF_API, CF_ORG, CF_SPACE, CF_USERNAME, and CF_PASSWORD for our capire org. View them here.

For demonstration purposes I created a PR deployment (after merging this PR we only re-deploy for pushes on main).
Deployment + App Link: https://github.com/capire/samples/deployments/Staging
Workflow run: https://github.com/capire/samples/actions/runs/16520349169/job/46720555138

→ works

The workflow and actions are exactly the ones created by cds add github-actions except that I had to adapt the lines fetching the server logs for the microservices setup.

Small heads-up: I had to add @sap/cds-dk to devDependencies for npx cds up to find the cds executable. This blows up our package-lock.json quite a bit. If we want to keep it lean we could use npm add @sap/cds-dk in the deployment workflow, but most stakeholders prefer to also freeze @sap/cds-dk.

In the future, we could also offer the actions on the GitHub Actions marketplace, essentially reducing this PR to a single file of 46 LoC.

@swaldmann swaldmann marked this pull request as ready for review July 25, 2025 11:20
@swaldmann swaldmann requested a review from danjoa July 25, 2025 11:20
@swaldmann swaldmann requested a review from Akatuoro July 25, 2025 14:22
@swaldmann swaldmann merged commit 6ac5b81 into main Jul 31, 2025
3 checks passed
@Akatuoro
Copy link
Contributor

In the future, we could also offer the actions on the GitHub Actions marketplace, essentially reducing this PR to a single file of 46 LoC.

I think that's a great idea

@swaldmann swaldmann deleted the gha branch July 31, 2025 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants